-
Notifications
You must be signed in to change notification settings - Fork 49
Add launch test log file support for the xpu test utils #1879
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Add launch test log file support for the xpu test utils #1879
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds file logging support to the XPU test utilities to help debug test execution, particularly in automation scenarios where tests may timeout and kill workflows. The logging captures command line parameters and execution details that can be used to reproduce failed test cases.
- Adds a file logger creation utility with configurable output paths
- Integrates logging into the
launch_test
function to record test commands - Supports flexible log file placement via environment variable or default location
Could you provide some log examples when there are hang and core dump? Why not put the log file under test/xpu? |
Here is one of the example log. The changes can provided more detail command line parameters and the next following test case. We can reuse the command in the log to directly reproduce the issue. ` ` |
Description
Add a file logger for helping record the launch test process. The log file can be used during the debugging. One of the scenario is in an automation test, the test workflow will be killed after the timeout. It is hard to determine whether is the current test failed lead to the issue or the next following cases. The current pytest output is not contain all the necessary command line parameters to rerun the failed case.
Changes